Skip to content

Fix proxy response handling and improve error handling#173

Open
chriswritescode-dev wants to merge 24 commits intomainfrom
feature/tui
Open

Fix proxy response handling and improve error handling#173
chriswritescode-dev wants to merge 24 commits intomainfrom
feature/tui

Conversation

@chriswritescode-dev
Copy link
Copy Markdown
Owner

Summary

  • Convert response body to text before creating new Response in proxy services
  • Use Set for header filtering to improve performance
  • Update error handling in useSendPrompt to use FetchError
  • Add dev-only logging for SSE initial session data failures

Why

  • Fix proxy response handling to properly consume response body
  • Improve performance with Set-based header filtering
  • Better error handling with typed FetchError

Type Of Change

  • Bug fix
  • Refactoring

Checklist

  • Code follows project style
  • Types updated
  • pnpm lint passes

- Introduce <promise>DONE</promise> completion signal for architect and code agents
- Require verification steps in all plans with mandatory test/lint/type-check commands
- Add session status fetching to CLI status command
- Improve session output formatting with recent activity display
- Fix v2 client session status calls to include worktree directory parameter
- Bump version to 0.0.28
- Add useWorkspaces config option for loop worktree management
- Implement SQLite-based loop state persistence in TUI
- Update build script to use Bun.build with solid plugin
- Improve loop state handling for workspace vs worktree detection
- Add click-to-select functionality for loops in sidebar
- Update tests to use wrk_loop_ workspace ID prefix
…lling

- Replace zodResolver with standardSchemaResolver in 10 frontend files to fix zod version incompatibility
- Add @standard-schema/spec dev dependency
- Improve loop polling logic in tui.tsx
- Add error logging for toast failures in loop.ts
Extract tool definitions from 1554-line index.ts into focused modules under tools/:
- tools/types.ts: Shared ToolContext interface and types
- tools/memory.ts: Memory CRUD tools (read, write, edit, delete)
- tools/kv.ts: Key-value store tools
- tools/health.ts: Health check and reindex tools
- tools/plan-execute.ts: Plan execution tool
- tools/loop.ts: Memory loop tools with setup orchestration
- tools/plan-approval.ts: Plan approval directives and loop blocking hooks
- tools/index.ts: Barrel export with createTools factory

Results:
- index.ts reduced from 1554 to 321 lines (79% reduction)
- Factory pattern: createXxxTools(ctx: ToolContext) for each category
- ToolContext interface consolidates all dependencies
- No runtime behavior changes - pure refactoring
- All 295 tests passing, lint and build successful
- Convert response body to text before creating new Response in proxy services
- Use Set for header filtering to improve performance
- Update error handling in useSendPrompt to use FetchError
- Add dev-only logging for SSE initial session data failures
- Replace backtick-wrapped tool names with "tool" suffix for clarity
- Add memory-kv-delete to librarian's excluded tools list
- Update auditor agent instructions for memory KV operations
- Fix loop service review instructions for consistency
Move memory plugin installation from Dockerfile build-time to entrypoint runtime.
Add INSTALL_MEMORY_PLUGIN env var (defaults to true) to control installation.
Update Dockerfile, docker-compose.yml, entrypoint script, and .env.example.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant